Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change llvmcall docs binding #16332

Merged
merged 1 commit into from
May 13, 2016

Conversation

MichaelHatherly
Copy link
Member

The docstring for llvmcall was bound to a keyword rather than the actual intrinsic Core.Intrinsics.llvmcall.

This produced the following odd behaviour when trying to find the docs:

help?> llvmcall

... docs ...

julia> llvmcall
ERROR: UndefVarError: llvmcall not defined

help?> Base.llvmcall

... no docs ...

julia> Base.llvmcall
(intrinsic function #90)

@MichaelHatherly MichaelHatherly added the docs This change adds or pertains to documentation label May 12, 2016
@tkelman
Copy link
Contributor

tkelman commented May 12, 2016

AppVeyor failure in abstractarray looks new, probably the fault of #16251 cc @mbauman

https://ci.appveyor.com/project/JuliaLang/julia/build/1.0.1501/job/ia6bwd91kg621ii3

@mbauman
Copy link
Member

mbauman commented May 12, 2016

Fix is very straightforward: acafa9b; caused by concurrent changes to similar.

@tkelman
Copy link
Contributor

tkelman commented May 12, 2016

Thanks much. I'll have to make barosl/homu#87 a project.

The docstring for `llvmcall` was bound to a keyword rather than
the actual intrinsic `Core.Intrinsics.llvmcall`.

This produced the following odd behaviour when trying to find the docs:

    help?> llvmcall

    ... docs ...

    julia> llvmcall
    ERROR: UndefVarError: llvmcall not defined

    help?> Base.llvmcall

    ... no docs ...

    julia> Base.llvmcall
    (intrinsic function JuliaLang#90)
@MichaelHatherly MichaelHatherly merged commit 906e06b into JuliaLang:master May 13, 2016
@MichaelHatherly MichaelHatherly deleted the mh/llvmcall-doc branch May 13, 2016 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants